Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Updated deleting & reconnecting calendars API integration. #540

Merged
merged 2 commits into from Oct 29, 2021
Merged

Updated deleting & reconnecting calendars API integration. #540

merged 2 commits into from Oct 29, 2021

Conversation

ghost
Copy link

@ghost ghost commented Oct 29, 2021

When connecting a calendar while there is a pre-existing connected calendar, the UI shows the newly connected calendar as the primary and gives option to remove it. Previously, it would still show the previous calendar as primary.

And when removing a calendar, the UI shows the POPUP_STAGES.CONNECT_CALENDAR modal view if there are no remaining connected calendars. But if there is, the UI shows the POPUP_STAGES.MANAGE_CALENDAR view. Previously, it would show the user POPUP_STAGES.MANAGE_AVAILABILITY view.

The changes in this PR are associated with issue.

When connecting a calendar while there is a pre-existing connected calendar, the UI shows the newly connected calendar as the primary and gives option to remove it. Previously, it would still show the previous calendar as primary.

And when removing a calendar, the UI shows the POPUP_STAGES.CONNECT_CALENDAR modal view if there are no remaining connected calendars. But if there is, the UI shows the POPUP_STAGES.MANAGE_CALENDAR view. Previously, it would show the user POPUP_STAGES.MANAGE_AVAILABILITY view.
Copy link
Contributor

@maxceem maxceem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you, @mahidulalvi-bonic, all works good.

@maxceem maxceem changed the base branch from feature/interview-update to dev October 29, 2021 09:54
@maxceem maxceem merged commit 5574436 into topcoder-archive:dev Oct 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants